-
Notifications
You must be signed in to change notification settings - Fork 457
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[DO NOT MERGE] Test support warning (re. assistance-only support and EOL) future states #18615
base: main
Are you sure you want to change the base?
Conversation
Files changed:
|
✅ Deploy Preview for cockroachdb-api-docs canceled.
|
✅ Deploy Preview for cockroachdb-interactivetutorials-docs canceled.
|
✅ Netlify Preview
To edit notification comments on pull requests, go to your Netlify site configuration. |
@mdlinville When I manually set
But shouldn't a warning also appear at the top of every doc page in 23.1 and 23.2? Nothing's showing up here:
Am I missing something, or is there a bug? Note: The reason I continued to set a variable
to validate the assignment of each. But I didn't need to do that, because the latest |
Can you rerun the test and instead of changing |
- Also add debug messages, disabled by default
I think this is ready to test again. When we are satisfied, we should decouple the logic changes and put them in another PR. |
A test, following on #18611